Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add HTTP and gRPC error codes retries to the resiliency policies documentation #4363

Open
wants to merge 10 commits into
base: v1.14
Choose a base branch
from
Open

Add HTTP and gRPC error codes retries to the resiliency policies documentation #4363

wants to merge 10 commits into from
+22 −0

Conversation

antontroshin
Copy link
Contributor

@antontroshin antontroshin commented Sep 19, 2024 •
edited
Loading

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within codetabs
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:

Description

Issue reference

Issue: #3859

@antontroshin antontroshin requested review from a team as code owners September 19, 2024 22:56
@github-actions GitHub Actions
Copy link

Stale PR, paging all reviewers

@github-actions github-actions bot added the stale label Sep 25, 2024
@github-actions github-actions bot closed this Oct 2, 2024
@yaron2 yaron2 reopened this Oct 2, 2024
@hhunter-ms hhunter-ms removed the stale label Oct 3, 2024
hhunter-ms
hhunter-ms requested changes Oct 3, 2024
View reviewed changes
Copy link
Collaborator

@hhunter-ms hhunter-ms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

quick review - thanks for your contribution!

daprdocs/content/en/operations/resiliency/policies.md Outdated Show resolved Hide resolved
daprdocs/content/en/operations/resiliency/policies.md Outdated Show resolved Hide resolved
daprdocs/content/en/operations/resiliency/policies.md Outdated Show resolved Hide resolved
msfussell
msfussell reviewed Oct 3, 2024
View reviewed changes
Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hhunter-ms - PTAL.

msfussell and others added 5 commits October 3, 2024 13:44
msfussell
msfussell approved these changes Oct 3, 2024
View reviewed changes
Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions GitHub Actions
Copy link

Stale PR, paging all reviewers

@github-actions github-actions bot added the stale label Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hhunter-ms hhunter-ms hhunter-ms requested changes

@msfussell msfussell msfussell approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

@antontroshin antontroshin

Labels
stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@antontroshin @msfussell @hhunter-ms @yaron2